-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Multiplatform #520
Multiplatform #520
Conversation
b9062b2
to
5a469b0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this just cleanup of unnecessary dependencies post the move to model-specific python dependency files a while back?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After moving to 3.10 we didn't need imporlib-metadata anymore and because the python installs are now split symfit moved to 3.8. I think upgrading to 3.10 also allowed the matplotlib pinning to be skipped. It's still installed though, just though a different dependency.
with: | ||
registry: ${{ env.REGISTRY }} | ||
username: ${{ github.actor }} | ||
password: ${{ secrets.GITHUB_TOKEN }} | ||
|
||
- name: Set up QEMU |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can i get a quick update on what QEMU is doing?
QEMU is required in order to build the arm64 image. Eventually we might be
able to do the build on an arm64 worker, but for now it's emulated
…On Tue, Aug 6, 2024, 06:43 Anya Petersen ***@***.***> wrote:
***@***.**** approved this pull request.
------------------------------
On pyproject.toml
<#520 (comment)>:
is this just cleanup of unnecessary dependencies post the move to
model-specific python dependency files a while back?
------------------------------
In .github/workflows/ci.yml
<#520 (comment)>:
> with:
registry: ${{ env.REGISTRY }}
username: ${{ github.actor }}
password: ${{ secrets.GITHUB_TOKEN }}
+ - name: Set up QEMU
can i get a quick update on what QEMU is doing?
—
Reply to this email directly, view it on GitHub
<#520 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABDNI6O5TJEXWY7XYMTCIVTZQDAIFAVCNFSM6AAAAABLQ4CGACVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDEMRRGE4DQMJWG4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
alfalfa-dependencies
linux/amd64
andlinux/arm64
Size can be shrunk further by removing pre-installed python 3.8 dependencies which are just there to make the
refrig-case
test run faster. If we used this test less we could remove those as default installed dependencies and reduce the size by another ~400MB.